Skip to content

ci: fix integration test#20

Merged
smacker merged 4 commits intomasterfrom
bzz-patch-ci
Dec 11, 2018
Merged

ci: fix integration test#20
smacker merged 4 commits intomasterfrom
bzz-patch-ci

Conversation

@bzz
Copy link
Copy Markdown
Contributor

@bzz bzz commented Dec 11, 2018

Most probably related to src-d/lookout#378

Fixes CI failure from #19

Copy link
Copy Markdown
Contributor

@carlosms carlosms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍, the -v flag was removed in v0.6.0

@carlosms
Copy link
Copy Markdown
Contributor

Although it might be good to add --log-level=debug

Copy link
Copy Markdown
Contributor

@smacker smacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. we need to set debug log level and update readme also.

bzz added 3 commits December 11, 2018 13:52
Replace `-v` with new debug log flag `--log-level=debug` in `lookout-sdk`
Copy link
Copy Markdown
Contributor

@smacker smacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@smacker smacker merged commit 3e92222 into master Dec 11, 2018
@bzz bzz deleted the bzz-patch-ci branch December 11, 2018 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants